Skip to content

[Init]: Boilerplate scaffold removed#12

Merged
Marius Storhaug (MariusStorhaug) merged 4 commits intomainfrom
fix/11-remove-boilerplate-scaffold
May 7, 2026
Merged

[Init]: Boilerplate scaffold removed#12
Marius Storhaug (MariusStorhaug) merged 4 commits intomainfrom
fix/11-remove-boilerplate-scaffold

Conversation

@MariusStorhaug
Copy link
Copy Markdown
Member

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented May 7, 2026 •
edited
Loading

The module repository no longer contains placeholder template code from the PSModule framework scaffolding. Only the New-Jwt placeholder function remains, signaling that the module is ready for actual JWT implementation work.

Changed: Repository content

All template/demo artifacts are removed — classes (Book, SecretWriter), functions (Get-PSModuleTest, New-PSModuleTest, Set-PSModuleTest, Test-PSModuleTest), variables (Planets, Moons, SolarSystems), data files, format definitions, type extensions, init scripts, nested modules, script loaders, examples, and the manifest override file.

What remains in src/:

src/
  functions/
    public/
      New-Jwt.ps1

The New-Jwt function emits a warning that it is not yet implemented. This gives CI a valid function to build and test against while development proceeds in #3.

Changed: Test file

The test file is renamed from PSModuleTest.Tests.ps1 to Jwt.Tests.ps1 with a single test verifying the placeholder warning output.

Changed: README

Rewritten to describe the Jwt module — name, purpose, installation, and current development status.

Technical Details

  • Removed 32 files totaling ~660 lines of template code
  • Code coverage target in .github/PSModule.yml lowered from 50% to 0% — the placeholder function has no meaningful logic to cover
  • The manifest.psd1 is removed because it is optional in the PSModule framework (only needed to override generated values); will be recreated in Implement JWT creation and signing functionality #3 when module metadata is defined
  • This PR completes the "Skeleton cleanup" section of the implementation plan in Implement JWT creation and signing functionality #3

@MariusStorhaug Marius Storhaug (MariusStorhaug) added the Patch Small fixes or improvements label May 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Fail ❌
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

Super-linter detected linting errors

For more information, see the GitHub Actions workflow run

Powered by Super-linter

POWERSHELL 

�[32;1mRuleName                           �[0m�[32;1m Severity    �[0m�[32;1m ScriptName�[0m�[32;1m Line �[0m�[32;1m Message�[0m
�[32;1m--------                           �[0m �[32;1m--------    �[0m �[32;1m----------�[0m �[32;1m---- �[0m �[32;1m-------�[0m
PSUseShouldProcessForStateChangingF Warning      New-Jwt.ps 1     Function 'New
unctions                                         1                -Jwt' has ver
                                                                  b that could
                                                                  change system
                                                                   state. There
                                                                  fore, the fun
                                                                  ction has to
                                                                  support 'Shou
                                                                  ldProcess'.

@MariusStorhaug Marius Storhaug (MariusStorhaug) added NoRelease Documentation, maintenance, CI/CD — no version bump and removed Patch Small fixes or improvements labels May 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Fail ❌
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

Super-linter detected linting errors

For more information, see the GitHub Actions workflow run

Powered by Super-linter

POWERSHELL 

�[32;1mRuleName                           �[0m�[32;1m Severity    �[0m�[32;1m ScriptName�[0m�[32;1m Line �[0m�[32;1m Message�[0m
�[32;1m--------                           �[0m �[32;1m--------    �[0m �[32;1m----------�[0m �[32;1m---- �[0m �[32;1m-------�[0m
PSUseShouldProcessForStateChangingF Warning      New-Jwt.ps 1     Function 'New
unctions                                         1                -Jwt' has ver
                                                                  b that could
                                                                  change system
                                                                   state. There
                                                                  fore, the fun
                                                                  ction has to
                                                                  support 'Shou
                                                                  ldProcess'.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Fail ❌
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

Super-linter detected linting errors

For more information, see the GitHub Actions workflow run

Powered by Super-linter

POWERSHELL 

�[32;1mRuleName                           �[0m�[32;1m Severity    �[0m�[32;1m ScriptName�[0m�[32;1m Line �[0m�[32;1m Message�[0m
�[32;1m--------                           �[0m �[32;1m--------    �[0m �[32;1m----------�[0m �[32;1m---- �[0m �[32;1m-------�[0m
PSShouldProcess                     Warning      New-Jwt.ps 19    'New-Jwt' has
                                                 1                 the ShouldPr
                                                                  ocess attribu
                                                                  te but does n
                                                                  ot call Shoul
                                                                  dProcess/Shou
                                                                  ldContinue.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Fail ❌
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

Super-linter detected linting errors

For more information, see the GitHub Actions workflow run

Powered by Super-linter

POWERSHELL 

�[32;1mRuleName                           �[0m�[32;1m Severity    �[0m�[32;1m ScriptName�[0m�[32;1m Line �[0m�[32;1m Message�[0m
�[32;1m--------                           �[0m �[32;1m--------    �[0m �[32;1m----------�[0m �[32;1m---- �[0m �[32;1m-------�[0m
PSShouldProcess                     Warning      New-Jwt.ps 19    'New-Jwt' has
                                                 1                 the ShouldPr
                                                                  ocess attribu
                                                                  te but does n
                                                                  ot call Shoul
                                                                  dProcess/Shou
                                                                  ldContinue.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@MariusStorhaug Marius Storhaug (MariusStorhaug) changed the title 🩹 [Patch]: Boilerplate scaffold removed [Init]: Boilerplate scaffold removed May 7, 2026
@MariusStorhaug Marius Storhaug (MariusStorhaug) marked this pull request as ready for review May 7, 2026 17:10
Copilot AI review requested due to automatic review settings May 7, 2026 17:10
Copilot AI reviewed May 7, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR strips the repository down from the PSModule scaffold/template artifacts so the Jwt module is ready for real JWT implementation work, while keeping a single New-Jwt placeholder plus a minimal test and updated documentation.

Changes:

  • Removed scaffold/template classes, functions, variables, formats/types, init/scripts, examples, and manifest override file.
  • Added a New-Jwt placeholder function and a single Pester test validating its “not yet implemented” warning.
  • Updated README and lowered CI code coverage target to 0% during the placeholder phase.

Reviewed changes

Copilot reviewed 31 out of 32 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/PSModuleTest.Tests.ps1 Removed scaffold Pester tests targeting PSModuleTest demo functions.
tests/Jwt.Tests.ps1 Added minimal Pester test asserting New-Jwt emits an “not yet implemented” warning.
src/variables/public/SolarSystems.ps1 Removed scaffold public variable data.
src/variables/public/Planets.ps1 Removed scaffold public variable data.
src/variables/public/Moons.ps1 Removed scaffold public variable data.
src/variables/private/PrivateVariables.ps1 Removed scaffold private variable data.
src/types/FileInfo.Types.ps1xml Removed scaffold type extension.
src/types/DirectoryInfo.Types.ps1xml Removed scaffold type extension.
src/scripts/loader.ps1 Removed scaffold script loader.
src/README.md Removed scaffold internal src README.
src/modules/OtherPSModule.psm1 Removed scaffold nested module example.
src/manifest.psd1 Removed optional scaffold manifest override file.
src/init/initializer.ps1 Removed scaffold init script.
src/header.ps1 Removed scaffold header script.
src/functions/public/Test-PSModuleTest.ps1 Removed scaffold public function.
src/functions/public/Set-PSModuleTest.ps1 Removed scaffold public function.
src/functions/public/New-PSModuleTest.ps1 Removed scaffold public function and aliases.
src/functions/public/New-Jwt.ps1 Added New-Jwt placeholder function that emits a warning.
src/functions/public/Get-PSModuleTest.ps1 Removed scaffold public function.
src/functions/private/Set-InternalPSModule.ps1 Removed scaffold private function.
src/functions/private/Get-InternalPSModule.ps1 Removed scaffold private function.
src/formats/Mygciview.Format.ps1xml Removed scaffold format view definition.
src/formats/CultureInfo.Format.ps1xml Removed scaffold format view definition.
src/finally.ps1 Removed scaffold “finally” script.
src/data/Settings.psd1 Removed scaffold data file.
src/data/Config.psd1 Removed scaffold data file.
src/classes/public/Book.ps1 Removed scaffold demo classes/enums.
src/classes/private/SecretWriter.ps1 Removed scaffold demo class.
README.md Rewritten to describe the Jwt module and current placeholder status.
examples/General.ps1 Removed scaffold example script.
.github/PSModule.yml Lowered code coverage target to 0% for the placeholder stage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/functions/public/New-Jwt.ps1
New-Jwt
```

Returns a placeholder string indicating that the function is not yet implemented.
Comment thread src/functions/public/New-Jwt.ps1
Comment on lines +19 to +27
[CmdletBinding(SupportsShouldProcess)]
param()

begin {}

process {
if ($PSCmdlet.ShouldProcess('Creating a new JWT')) {
Write-Warning 'New-Jwt is not yet implemented.'
}
@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit dde4b24 into main May 7, 2026
65 of 67 checks passed
@MariusStorhaug Marius Storhaug (MariusStorhaug) deleted the fix/11-remove-boilerplate-scaffold branch May 7, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@MariusStorhaug Marius Storhaug (MariusStorhaug)

Labels

NoRelease Documentation, maintenance, CI/CD — no version bump

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove boilerplate scaffold and prepare repository for Jwt development

2 participants

@MariusStorhaug